[Benchmarks] Update benchmarks README.md #18954
Conversation
| are stored [here](https://oneapi-src.github.io/unified-runtime/performance/). | ||
|
|
||
| ## Output formats | ||
| ### Output formats |
There was a problem hiding this comment.
is the extra # intentional?
There was a problem hiding this comment.
Yes, I think this fits into the subsection of “Results”.
devops/scripts/benchmarks/README.md
Outdated
| ## Contribution | ||
|
|
||
| The requirements and instructions above are for building the project from source | ||
| without any modifications. To make modifications to the specification, please see the | ||
| [Contribution Guide](https://oneapi-src.github.io/unified-runtime/core/CONTRIB.html) | ||
| for more detailed instructions on the correct setup. |
There was a problem hiding this comment.
is the UR CONTRIB actually relevant for benchmark scripts?
There was a problem hiding this comment.
I think it can be dropped
There was a problem hiding this comment.
Do you think we should remove the entire Contribution section?
There was a problem hiding this comment.
I think the section could be removed as there is a CONTRIB.md file in the same directory with a description of contributing to benchmarks framework. Contributions to Unified Runtime are irrelevant to the benchmarks framework
There was a problem hiding this comment.
I decided to leave this section because it finishes the README nicely. I've updated the link to the benchmarks CONTRIB.
|
@intel/llvm-gatekeepers, please merge |
Can you please provide |
|
Sorry. Accidentally pressed all the wrong buttons. |
Preview: https://github.com/wlemkows/llvm/tree/readme-update/devops/scripts/benchmarks